add docs for check_and_build_global_config()
#270
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yucongalicechen
commented
Dec 23, 2024
yucongalicechen
commented
Dec 23, 2024
yucongalicechen
commented
Dec 23, 2024
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #270 +/- ##
=======================================
Coverage 98.68% 98.68%
=======================================
Files 8 8
Lines 379 379
=======================================
Hits 374 374
Misses 5 5 |
yucongalicechen
commented
Dec 23, 2024
|
|
||
|
|
||
| def stringify(obj): | ||
| def _stringify(obj): |
There was a problem hiding this comment.
make this function private and remove unncessary functions
sbillinge
reviewed
Dec 23, 2024
| Please enter your orcid ID if you know it: 0000-0000-0000-0000 | ||
|
|
||
|
|
||
| After receiving the inputs, the function will write the following to the file: |
There was a problem hiding this comment.
Maybe ".... the diffpyconfig.json file in the users home directory"?
On the one hand it is a bit repetitive (also did elsewhere), but maybe it is better to be clear?
| Developers can use this function to simplify the process of loading, merging, and saving information consistently and easily. | ||
| Additionally, it saves the effort of re-entering information, and allows overriding current information by | ||
| passing parameters. | ||
|
|
||
| - ``check_and_build_global_config()``: This function helps create a global configuration file | ||
| that can be used by ``get_user_info()``. | ||
| If no existing configuration file is found, and the user allows inputs, this function prompts for information. |
There was a problem hiding this comment.
I think "user allows inputs" is confusing so I recommend to delete that.
yucongalicechen
commented
Dec 23, 2024
yucongalicechen
commented
Dec 23, 2024
|
@sbillinge ready for review! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #268
@sbillinge ready for review